Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

macOS screen share audio #561

Merged
merged 69 commits into from
Feb 12, 2025
Merged

macOS screen share audio #561

merged 69 commits into from
Feb 12, 2025

Conversation

hiroshihorie
Copy link
Member

No description provided.

Base automatically changed from hiroshi/adm-audioengine2 to main January 30, 2025 16:43
@hiroshihorie hiroshihorie requested a review from bcherry February 6, 2025 11:26
@hiroshihorie hiroshihorie marked this pull request as ready for review February 6, 2025 11:29
@ladvoc
Copy link
Contributor

ladvoc commented Feb 11, 2025

While testing this in the example app, I came across an issue where screen share audio still plays on the remote participant's device even after screen sharing is stopped.

Steps to reproduce:

  1. Join a room using both a Mac and another device.
  2. Play an audio source on the Mac.
  3. Enable screen sharing on the Mac, then enable the microphone to publish an audio track.
  4. Wait ~20 seconds, then disable screen sharing.
  5. Stop the audio source on the Mac.
  6. Re-enable screen sharing, and listen to the audio from the second device—you will still hear the previous screen share audio even though it is no longer playing.

@hiroshihorie
Copy link
Member Author

@ladvoc I think the audio from the screen share picker was being captured and causing confusion.
Can you try this branch: livekit-examples/swift-example#64
I also changed capturing app audio to false by default.

@ladvoc
Copy link
Contributor

ladvoc commented Feb 11, 2025

Yep, that fixed it.

@hiroshihorie hiroshihorie merged commit 485e76d into main Feb 12, 2025
11 of 12 checks passed
@hiroshihorie hiroshihorie deleted the hiroshi/mac-screenshare-audio branch February 12, 2025 05:36
hiroshihorie added a commit to ladvoc/client-sdk-swift that referenced this pull request Feb 13, 2025
commit aa0a21d
Author: Jacob Gelman <[email protected]>
Date:   Thu Feb 13 06:28:21 2025 -0800

    Simplify broadcast extension setup with standard format for identifiers (livekit#573)

    When configuring a broadcast extension, manually setting the info keys
    `RTCAppGroupIdentifier` and `RTCScreenSharingExtension` is no longer
    required when using the standard format. The standard format is as
    follows:
    - App group: `group.<main-app-bundle-id>`
    - Broadcast extension: `<main-app-bundle-id>.broadcast`

    ---------

    Co-authored-by: Hiroshi Horie <[email protected]>

commit c3ee701
Author: Jacob Gelman <[email protected]>
Date:   Thu Feb 13 06:05:00 2025 -0800

    Deprecate public broadcast picker extension (livekit#586)

    Public show method defined as an extension to
    `RPSystemBroadcastPickerView` has been deprecated in favor of
    `BroadcastManager.shared.requestActivation()`.

    ---------

    Co-authored-by: Hiroshi Horie <[email protected]>

commit 0da6660
Author: Jacob Gelman <[email protected]>
Date:   Thu Feb 13 05:55:25 2025 -0800

    Release automation (livekit#579)

    - Add version and platform compatibility badges from [Swift Package
    Index](https://swiftpackageindex.com/) to README
      - Automatically updated on each release
    - Add [nanpa](https://github.com/nbsp/nanpa) configuration
    - Custom script bumps version across repo (currently Podspec, README,
    and LiveKitSDK class)
      - GitHub publish workflow (based on workflow from livekit/rust-sdks)
    - Create workflow to push new releases to Cocopods when a release is
    published on GitHub

    I have also added changeset files to my currently open PRs (livekit#565, livekit#576,
    and livekit#573) that can be used to test this configuration.

    ---------

    Co-authored-by: Hiroshi Horie <[email protected]>

commit 5b031c8
Author: Hiroshi Horie <[email protected]>
Date:   Thu Feb 13 22:36:35 2025 +0900

    Update Podspec (livekit#587)

    Fixes: livekit#566

commit 485e76d
Author: Hiroshi Horie <[email protected]>
Date:   Wed Feb 12 14:36:48 2025 +0900

    macOS screen share audio (livekit#561)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants